I18nTextDomainFixer: remove unnecessary variable initialization #2513
Conversation
Initializing the variable here is unnecessary as its value is immediately overridden in the if/else below.
|
We could probably run 'inspect code' function from PhpStorm and quickly see similar unused variables in the codebase (and potential other fixes from the suggestion) 
|
That is a good idea, @dingo-d. If you are not planning to do it yourself, I can try that. Just let me know what you prefer. |
|
I mean if you're already working on it you can take over, but it's not a problem for me to do as well 😄 |
|
Sounds like a plan gentlemen ;-) Please do check history though when doing this as some of these things may have been put in place to satisfy a higher PHPStan level than we're currently running at and a commit message may hint at that, if that's the case. |
|
Just documenting here that I created the PR to remove unused variables detected by PHPStorm: #2514 |
While working on #2512, I noticed an unnecessary variable initialization, so I'm creating this PR to suggest that it be removed. Its value is immediately overridden in an if/else below.
https://github.com/rodrigoprimo/WordPress-Coding-Standards/blob/690f05a51d190586e0bb0426a0f04242224d0723/WordPress/Sniffs/Utils/I18nTextDomainFixerSniff.php#L727
https://github.com/rodrigoprimo/WordPress-Coding-Standards/blob/690f05a51d190586e0bb0426a0f04242224d0723/WordPress/Sniffs/Utils/I18nTextDomainFixerSniff.php#L749